Skip to content

Add Preview button to String Substitution preference page - #3062

Open
SougandhS wants to merge 1 commit into
eclipse-platform:masterfrom
SougandhS:PreviewVal
Open

SougandhS wants to merge 1 commit into
eclipse-platform:masterfrom
SougandhS:PreviewVal

Conversation

@SougandhS

Copy link
Copy Markdown
Contributor

This commit adds a "Preview" button on the String Substitution pref page. It is enabled for a single selection and opens a small dialog showing the variable's name, value, description and contributing plug-in, each with a bold title. This will be useful for readability when there are lengthy values

image

@github-actions

github-actions Bot commented Nov 8, 2026 •

Copy link
Copy Markdown
Contributor

Test Results

    54 files  ±0      54 suites  ±0   38m 24s ⏱️ + 4m 16s
 5 005 tests ±0   4 983 ✅ ±0   22 💤 ±0  0 ❌ ±0 
12 906 runs  ±0  12 752 ✅ ±0  154 💤 ±0  0 ❌ ±0 

Results for commit afc6fa7. ± Comparison against base commit 5cccf6a.

♻️ This comment has been updated with latest results.

@SougandhS
SougandhS force-pushed the PreviewVal branch 4 times, most recently from c492818 to cc327f1 Compare November 9, 2026 09:02
This commit adds a "Preview" button on the String Substitution pref
page. It is enabled for a single selection and opens a small
dialog showing the variable's name, value, description and contributing
plug-in, each with a bold title. This will be useful for readability
when there is long values

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🟢 Approval recommended

The String Substitution preview change has no active review issues.

No open issues 🎉

What changed in this PR

Adds a Preview action to the String Substitution preference page, making lengthy variable values easier to read without editing them.

Changes:

  • Enables Preview when exactly one variable is selected.
  • Adds a resizable, read-only dialog with bold headings for variable details and contributing plug-in information.
  • Adds localized labels and simplifies existing iterator loops.
File Description
debug/​org.eclipse.debug.ui/​ui/​org/​eclipse/​debug/​internal/​ui/​preferences/​StringVariablePreviewDialog.java Implements the formatted variable preview dialog.
debug/​org.eclipse.debug.ui/​ui/​org/​eclipse/​debug/​internal/​ui/​preferences/​StringVariablePreferencePage.java Adds Preview button handling and simplifies loops.
debug/​org.eclipse.debug.ui/​ui/​org/​eclipse/​debug/​internal/​ui/​preferences/​DebugPreferencesMessages.properties Defines localized preview labels.
debug/​org.eclipse.debug.ui/​ui/​org/​eclipse/​debug/​internal/​ui/​preferences/​DebugPreferencesMessages.java Declares the corresponding message fields.

Review effort: Balanced

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants